Skip to content
This repository has been archived by the owner on Sep 20, 2019. It is now read-only.

Put className on the element wrapper #326

Merged
merged 1 commit into from
Apr 2, 2019
Merged

Put className on the element wrapper #326

merged 1 commit into from
Apr 2, 2019

Conversation

sorvell
Copy link
Contributor

@sorvell sorvell commented Apr 1, 2019

In noPatch mode, the wrapper must be used to set className since it must preserve style scoping.

In `noPatch` mode, the wrapper must be used to set `className` since it must preserve style scoping.
sorvell pushed a commit to Polymer/polymer that referenced this pull request Apr 1, 2019
… used.

Puts `className` on `Polymer.dom` and ensures any sets to `className` in the binding system, go through that wrapper.

Note, depends on webcomponents/shadydom#326.
Copy link
Contributor

@dfreedm dfreedm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dfreedm dfreedm merged commit fdc2643 into master Apr 2, 2019
@dfreedm dfreedm deleted the wrap-className branch April 2, 2019 00:19
sorvell pushed a commit to Polymer/polymer that referenced this pull request Apr 2, 2019
… used.

Puts `className` on `Polymer.dom` and ensures any sets to `className` in the binding system, go through that wrapper.

Note, depends on webcomponents/shadydom#326.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants